Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix #394: Make import work with Terra dev version #395

Merged
merged 2 commits into from
Jun 23, 2022

Conversation

garrison
Copy link
Member

Summary

Fixes #394

Details and comments

@rathishcholarajan
Copy link
Member

@garrison Thanks, we should probably port this over here Qiskit/qiskit-ibmq-provider#1135 ?

@garrison
Copy link
Member Author

That would be a fine alternative to this, yes. It is a bit more complicated but might cover some corner cases that this does not. @mtreinish

@coveralls
Copy link

Pull Request Test Coverage Report for Build 2545475155

  • 2 of 2 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.008%) to 65.933%

Totals Coverage Status
Change from base Build 2539227711: 0.008%
Covered Lines: 2667
Relevant Lines: 4045

💛 - Coveralls

Copy link
Member

@mtreinish mtreinish left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be sufficient and should get what we need. I just took the regex from packaging for elsewhere because it handles grouping all the components of a pep440 compliant version string. But in this case we really don't need to worry about the dev version or a post release suffix since this is just used to tell which api to use.

@rathishcholarajan
Copy link
Member

Thank you @garrison!

@rathishcholarajan rathishcholarajan merged commit 315a088 into Qiskit:main Jun 23, 2022
@garrison garrison deleted the allow-terra-dev-version branch June 23, 2022 16:20
@rathishcholarajan rathishcholarajan added the Changelog: Bugfix Include in the Fixed section of the changelog label Jul 5, 2022
@rathishcholarajan rathishcholarajan added this to the 0.6.0 milestone Jul 5, 2022
blakejohnson pushed a commit to blakejohnson/qiskit-ibm-runtime that referenced this pull request May 26, 2023
* Fix flag error.

* Fix enum type conversion.

* Remove __int__ conversion>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changelog: Bugfix Include in the Fixed section of the changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Import fails with development version of Qiskit Terra
4 participants